Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add go-libp2p in 2022 blog entry #33

Merged
merged 11 commits into from
Feb 14, 2023
Merged

feat: add go-libp2p in 2022 blog entry #33

merged 11 commits into from
Feb 14, 2023

Conversation

p-shahi
Copy link
Member

@p-shahi p-shahi commented Jan 7, 2023

No description provided.

Copy link
Member

@salmad3 salmad3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

go go-libp2p ✨

src/_blog/go-libp2p-2022.md Outdated Show resolved Hide resolved
src/_blog/go-libp2p-2022.md Outdated Show resolved Hide resolved
src/_blog/go-libp2p-2022.md Outdated Show resolved Hide resolved
src/_blog/go-libp2p-2022.md Outdated Show resolved Hide resolved
src/_blog/go-libp2p-2022.md Outdated Show resolved Hide resolved
src/_blog/go-libp2p-2022.md Outdated Show resolved Hide resolved
src/_blog/go-libp2p-2022.md Outdated Show resolved Hide resolved
src/_blog/go-libp2p-2022.md Outdated Show resolved Hide resolved
src/_blog/go-libp2p-2022.md Outdated Show resolved Hide resolved
src/_blog/go-libp2p-2022.md Show resolved Hide resolved
Copy link
Member

@mxinden mxinden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

src/_blog/go-libp2p-2022.md Outdated Show resolved Hide resolved
src/_blog/go-libp2p-2022.md Show resolved Hide resolved
src/_blog/go-libp2p-2022.md Show resolved Hide resolved

### Simplify libp2p Setup Logic with Fx 🦄

In [v0.24.0](https://github.com/libp2p/go-libp2p/releases/tag/v0.24.0) go-libp2p included a change to use [Fx](https://github.com/uber-go/fx), a Go dependency injection library.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not really, this is not an outwards-facing change.

It just massively simplified how we construct our transports, and it will enable us to properly start and stop libp2p services in the future (see libp2p/go-libp2p#1993).

src/_blog/go-libp2p-2022.md Show resolved Hide resolved
src/_blog/go-libp2p-2022.md Show resolved Hide resolved
@p-shahi
Copy link
Member Author

p-shahi commented Feb 13, 2023

@marten-seemann @MarcoPolo may I get your review, I'll publish this today

@github-actions
Copy link
Contributor

Images automagically compressed by Calibre's image-actions

Compression reduced images by 18.5%, saving 134.78 KB.

Filename Before After Improvement Visual comparison
src/assets/go-2022.png 228.40 KB 175.44 KB -23.2% View diff
src/assets/rust-libp2p-2022-header.png 140.48 KB 106.35 KB -24.3% View diff
src/assets/webrtc_browser_to_server_diagram.png 357.73 KB 310.03 KB -13.3% View diff

44 images did not require optimisation.

@github-actions
Copy link
Contributor

Images automagically compressed by Calibre's image-actions

Compression reduced images by 16.4%, saving 81.83 KB.

Filename Before After Improvement Visual comparison
src/assets/rust-libp2p-2022-header.png 140.48 KB 106.35 KB -24.3% View diff
src/assets/webrtc_browser_to_server_diagram.png 357.73 KB 310.03 KB -13.3% View diff

45 images did not require optimisation.

@p-shahi p-shahi merged commit af4d213 into main Feb 14, 2023
@p-shahi p-shahi deleted the go-2022 branch February 14, 2023 00:01
BigLep added a commit that referenced this pull request Feb 17, 2023
Followup on #33 since I missed the review there

Areas of focus:
1. Making clear that this is the EngRes go-libp2p team.  I know there isn't another team currently, but we would love it if there was.
2. Clarifying QUIC heading
3. Moving some things up that were should be considered features I believe.
@BigLep BigLep mentioned this pull request Feb 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants